Skip to content

Fetch share source metadata in bulk - #62987

Merged
icewind1991 merged 4 commits into
masterfrom
unified-sharing-source-metadata
Aug 7, 2026
Merged

Fetch share source metadata in bulk#62987
icewind1991 merged 4 commits into
masterfrom
unified-sharing-source-metadata

Conversation

@icewind1991

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

  • Introduce a class for storing the share source metadata (currently displayName and icon)
  • Fetch just an ICacheEntry instead of an entire Node for getting file source name
  • Fetch source metadata in bulk instead of one-by-one on demand.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>

# Conflicts:
#	apps/files/lib/Sharing/Source/NodeShareSourceMetadata.php
@icewind1991 icewind1991 added this to the Nextcloud 35 milestone Aug 6, 2026
@icewind1991
icewind1991 requested a review from a team as a code owner August 6, 2026 17:14
@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Aug 6, 2026
@icewind1991
icewind1991 requested review from Altahrim, CarlSchwan, leftybournes, nfebe and salmart-dev and removed request for a team August 6, 2026 17:14
@icewind1991
icewind1991 force-pushed the unified-sharing-source-metadata branch 2 times, most recently from 9631b2c to 368ee06 Compare August 7, 2026 17:04
Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991
icewind1991 force-pushed the unified-sharing-source-metadata branch from 368ee06 to 2cc819a Compare August 7, 2026 20:45
@icewind1991
icewind1991 merged commit f8f6e27 into master Aug 7, 2026
216 checks passed
@icewind1991
icewind1991 deleted the unified-sharing-source-metadata branch August 7, 2026 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants